feat: フレーズのシーケンスIDとAudioBufferをViewから取得できるようにする #2869
Conversation
|
🚀 プレビュー用ページを作成しました 🚀 更新時点でのコミットハッシュ： |
There was a problem hiding this comment.
Pull request overview
This PR adds the ability to retrieve phrase sequence IDs and AudioBuffer from Vue components to support audio waveform display functionality.
Key Changes:
- Added
phraseSequenceIdsstate to track the mapping between phrases and their sequence IDs - Introduced callback-based mechanism to update sequence ID mappings when sequences are created or deleted
- Added
GET_SEQUENCE_AUDIO_BUFFERgetter to retrieve AudioBuffer from a sequence ID
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/store/type.ts | Defines new state property phraseSequenceIds, mutations for managing phrase sequence IDs (SET_PHRASE_SEQUENCE_IDS, SET_PHRASE_SEQUENCE_ID, DELETE_PHRASE_SEQUENCE_ID), and getter GET_SEQUENCE_AUDIO_BUFFER |
| src/store/singing.ts | Implements state initialization, callback-based updates to phraseSequenceIds during sequence lifecycle, mutations for managing phrase sequence IDs, and getter logic for retrieving AudioBuffer from sequences |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Hiroshiba
left a comment
Hiroshiba
left a comment
There was a problem hiding this comment.
LGTM！！
ロジック周り見てみて問題なさそうなことを確認しました！
あとは @sevenc-nanashi さん的に実装意義に違和感なければOKそう。
音声波形を表示する機能の実装のため
これってissueとかってありましたっけ 👀
なかったらなかったでOKで、関連するissueとかPRあったら追えるように紐づけしておきたいな、くらいの意図です！
| * フレーズの状態と再生されるシーケンスの状態を同期させる。 | ||
| * 不要なシーケンスを削除し、不足しているシーケンスを生成する。 | ||
| */ | ||
| const syncPhraseSequences = ( |
There was a problem hiding this comment.
（ただのコメントです）
ちゃんとわかってないんですが、この辺の関数とか別ファイルに移しちゃっても良いかもですね！
src/store/singing.tsってなってる今のコードを
src/store/singing/index.tsにして、
src/store/singing/hoge.tsファイル作ってそっちに移す的なのとか。
もちろんやるなら別PRのが良さそう！
|
@Hiroshiba |
sigprogramming
requested review from
sevenc-nanashi
and removed request for
sevenc-nanashi
|
たぶん問題ないと思うのでマージします！ |
3 participants
内容
フレーズのシーケンスIDとAudioBufferをView（Vueコンポーネント）から取得できるようにします。
音声波形を表示する機能の実装のために行います。
関連 Issue
その他